-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bump go to 1.22, k8s deps to 0.30.0 #176
bump go to 1.22, k8s deps to 0.30.0 #176
Conversation
var obj client.Object = evt.Object | ||
if obj == nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was needed to maintain the existing expectations of the API where a nil
object in the event would result in returning a default filter value
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #176 +/- ##
==========================================
+ Coverage 88.71% 88.88% +0.17%
==========================================
Files 17 18 +1
Lines 567 576 +9
==========================================
+ Hits 503 512 +9
Misses 51 51
Partials 13 13 ☔ View full report in Codecov by Sentry. |
Signed-off-by: everettraven <everettraven@gmail.com>
b6409b8
to
4d3d4c4
Compare
Squashed commits |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
/lgtm
bf5fa88
Description of the change:
handler
package so that handlers satisfy the updated controller-runtime APIs that use genericsMotivation for the change: